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Why Rev iew 
Improve Code Quality 
Spread Knowledge 
Repeatability 
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Preparing for a 
Review 
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Asthe requestor 
Stash 

Pull requests (by hp) 

https: //confluence. devlan.net/displav/HOWTO/Configure+Stash+to+Enforce+Workflows+and+PuU-*- Requests 
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As the requestor 

Instructions for building/verifying 
° automated tests (unittest) 

° basic instructions to verify 

Use Linters (static analysis) tools 
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As the requestor 


Readability 


, ^include <math.h> , . . . . 

humans i bave 4© read/ understand it. 

♦include <unistd.n> 


<netinet/in. h> 


avoid clever stuff.,. ; . 


♦define S(x,y,z)F x(F*d,F z s){ F t=y; t+=y; return t+=y; > 

**s+*,*)sli^*d*+*=s JS(v,*d«-*+=s/Unt w(int c){ return»p+*=c; > F W(F* 
d){ return sqrt (IllcLd) ) ; j^void .r(uint32 t w(u»24); w(u»16); w(u» 

° UOC'UmC^t*^ n^^rU ) Tin^V' ( S ) LLlPT r 3r*z,*f; void u(criar*s){ * 
s£Aw((u(l>s),*s)); > F GIF x.mt p)r for(z=p*46*f ; 12^*11 x*=.b){ for(D=-4; 
5>0; D*+)if(9-*z*+)< *P++=x; *P«+=D/2; *P**=0; > > return p; } void K(){ write 
(k'O.P-g); P=g; > F*d( K h= 0 ; D=le9; a(E.O); 0(E); 2lE)6A0>{t=(U2[Ql)/2[E|)W, 
(D=-t,h=P) ; for(i=H; P>i; i+=3H B(C f l); a(C.Q); b=H(E,C); (t=b*b-H(C,CMi<L?99 
: .6) )>=0&&0<(t=b<t?t*b:b-t)&&D>t6iM[)=t,h=i) ; } return h; > void Y(int N){ F*ft,*i 

B(Q.n); X(Q, le-4) ; o(p,Q); X(c,0); if(N<8H B(Q,n); X(Q,2«H(n,E)); a(E,Q); o(E,p); 
B(Q,p); Y(1*N); X(c,h-P?.8: .2) ; > for(i=M; L>i; ♦♦!){ Bl E, i) ; B(Q,p); d()-i||v(c,( 
h-P?.l:.5)*H(n,E)); > v(c,0.05); h-P| | (2[c]»=.3,cll&lrint(*p>nrint(llp] )&l]«=.2); 
> ) void Z(char*s){ K(); p+=4; u(s?s:"TADr) ; > void J(){ uint32 t c=-9; e=p; p=g; 
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How to review 
code 
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CHECK OUT 

BUILD 

RUN 
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How long 
60-90 minutes 
200-400 lines of code 
No skimming 
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Use Stash 
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Use Stash 

Comment code inline 


32 33 ciaaa KIRScrvico(PliatScrvicc) t 

33 - d ef lnit (salt, amdevlce, servicenamca , format -kcrPropertyLlatBinaryFormatvlO ) : 

34 - PliatService. init (aol£, aadcvicc, servicenamca, format > 

34 ♦ dcf init (aclf, amdevlce, eervicenames , format=jcCFPropcrtyLiatainaryFormat_vi_c, buildversion*Mone) » 


| Thomas C 

NNNNOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOO mooooooooorrrnreeeeeeee 

Reply - Create task • Like - 26 Jan 2015 




HP 

finger guns! bam bam! -« -v 

Reply - Edit - Delete - Create task - 


35 ♦ PI iat service. init (self, amdevicc, servicenamca, format, buildveraion-boildveraion ) 
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Be constructive 


66 * static unsigned int 

67 ♦ _get_M(void) { 

68 4 static unsigned int ns - 0; 

69 4 static dispatchoncet oocej 

70 4 dispatch_oncs(4oncs, *{ 

71 4 nach_tisicbese_iafo_data_t tmebase info; 

72 4 if (aach_tisttbase_info[4tiaebaae_info) — 0) { 

73 4 ns - 10000000 * tinobaaoinfo.dsnc* / tlmi>eee_lnfo. tuner; 


| Daniel 

1 do you have an extra 0 here convert nanoseconds to ms? 

Reply Create task DM • 24 Feb 2015 


Thomas C 

LMAO, that beats HP's off*by-one bugs :) 

Reply Delete Create task • Luce • 24 Feb 2015 
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Use Stash 


All comments should be addressed 
° Fix it 
° Fight it 

° Flag it (jira task) 

Ask questions when code is not clear or no documentation 
Avoid style comments 
° formatting (unless inconsistent) 

° bracket locations 
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Summary 
Use Stash 

Review in small chunks 
Address all comments 


UNCLASSIFIED//FOUO 


15 



Questions 
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